Skip to content

fix(flink): enable batch read it for flink source v2#18325

Merged
xushiyan merged 2 commits intoapache:masterfrom
HuangZhenQiu:flink-source-v2-it
Mar 22, 2026
Merged

fix(flink): enable batch read it for flink source v2#18325
xushiyan merged 2 commits intoapache:masterfrom
HuangZhenQiu:flink-source-v2-it

Conversation

@HuangZhenQiu
Copy link
Copy Markdown
Collaborator

Describe the issue this Pull Request addresses

Fix the bug in Flink Hudi source v2 in batch mode

Summary and Changelog

  1. Fix the bug in Flink Hudi source v2 in batch mode
  2. Update test cases to reflect the change

Impact

none

Risk Level

none

Documentation Update

none

Contributor's checklist

  • Read through contributor's guide
  • Enough context is provided in the sections above
  • Adequate tests were added if applicable

@github-actions github-actions bot added the size:S PR with lines of changes in (10, 100] label Mar 15, 2026
@HuangZhenQiu HuangZhenQiu force-pushed the flink-source-v2-it branch 2 times, most recently from d835741 to 8afc107 Compare March 16, 2026 04:35
@github-actions github-actions bot added size:M PR with lines of changes in (100, 300] and removed size:S PR with lines of changes in (10, 100] labels Mar 16, 2026
@HuangZhenQiu HuangZhenQiu force-pushed the flink-source-v2-it branch 2 times, most recently from ad2b01e to a5756a6 Compare March 20, 2026 13:23
@github-actions github-actions bot added size:L PR with lines of changes in (300, 1000] and removed size:M PR with lines of changes in (100, 300] labels Mar 20, 2026
@apache apache deleted a comment from hudi-bot Mar 20, 2026
@HuangZhenQiu HuangZhenQiu force-pushed the flink-source-v2-it branch 2 times, most recently from c17505d to 15e58b4 Compare March 21, 2026 03:41
Copy link
Copy Markdown
Member

@xushiyan xushiyan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good to go once CI passes

@github-actions github-actions bot added size:M PR with lines of changes in (100, 300] and removed size:L PR with lines of changes in (300, 1000] labels Mar 22, 2026
@hudi-bot
Copy link
Copy Markdown
Collaborator

CI report:

Bot commands @hudi-bot supports the following commands:
  • @hudi-bot run azure re-run the last Azure build

@xushiyan xushiyan merged commit f74bf3a into apache:master Mar 22, 2026
56 checks passed
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 68.46%. Comparing base (967e456) to head (ea6b709).
⚠️ Report is 12 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master   #18325      +/-   ##
============================================
- Coverage     68.46%   68.46%   -0.01%     
  Complexity    27472    27472              
============================================
  Files          2427     2427              
  Lines        132655   132659       +4     
  Branches      15994    15994              
============================================
+ Hits          90820    90822       +2     
- Misses        34784    34787       +3     
+ Partials       7051     7050       -1     
Flag Coverage Δ
common-and-other-modules 44.43% <100.00%> (+<0.01%) ⬆️
hadoop-mr-java-client 45.06% <ø> (-0.01%) ⬇️
spark-client-hadoop-common 48.21% <ø> (-0.01%) ⬇️
spark-java-tests 48.82% <ø> (-0.01%) ⬇️
spark-scala-tests 44.95% <ø> (+<0.01%) ⬆️
utilities 38.63% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
...va/org/apache/hudi/source/reader/BatchRecords.java 97.14% <100.00%> (ø)
...he/hudi/source/reader/HoodieSourceSplitReader.java 91.17% <100.00%> (+1.17%) ⬆️
...rg/apache/hudi/source/split/HoodieSourceSplit.java 100.00% <100.00%> (ø)

... and 6 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M PR with lines of changes in (100, 300]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants